[#294] Book card polish: accent title + metadata alignment#295
Merged
realproject7 merged 1 commit intomainfrom Mar 18, 2026
Merged
[#294] Book card polish: accent title + metadata alignment#295realproject7 merged 1 commit intomainfrom
realproject7 merged 1 commit intomainfrom
Conversation
- Title text uses text-accent (green) instead of text-foreground (white) - Metadata row left padding adjusted to pl-[7px] to align with front cover's visual left edge (6px layer offset + 1px border) Fixes #294 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
approved these changes
Mar 18, 2026
Collaborator
project7-interns
left a comment
project7-interns
left a comment
There was a problem hiding this comment.
Clean two-line change. Title color swap and metadata alignment math (6px offset + 1px border = 7px) both check out. LGTM.
project7-interns
approved these changes
Mar 18, 2026
Collaborator
project7-interns
left a comment
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
The PR stays within issue #294 scope and applies the two requested StoryCard polish changes cleanly.
Findings
- No blocking findings.
Decision
Approving because the title now uses the accent green and the metadata row padding accounts for the 6px page offset without broadening the change set.
realproject7
added a commit
that referenced
this pull request
Mar 27, 2026
Add checkUserExists() helper that checks if any DB row exists for a wallet. Agents page and dashboard now distinguish between known users with agent_id=NULL (definitive non-agents, skip RPC) and completely unknown wallets (still eligible for RPC fallback + auto-cache). Eliminates 3+ unnecessary RPC calls per visit for known non-agent users. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
realproject7
added a commit
that referenced
this pull request
Mar 27, 2026
Known non-agents (DB row with agent_id=NULL) now get a single non-blocking agentIdByWallet RPC call to catch external registrations. If positive, auto-caches and updates UI. Avoids the heavy 3-call chain (balanceOf, tokenOfOwnerByIndex, etc.) while still detecting users who registered as agents externally after their PlotLink account was created. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
realproject7
added a commit
that referenced
this pull request
Mar 27, 2026
Remove lightweight background agentIdByWallet check for known non-agents. Users with a DB row and agent_id=NULL now make zero RPC calls — definitively treated as non-agents per issue spec. External agent registrations for existing PlotLink users are handled via profile refresh flow. Full RPC fallback (3+ calls) only fires for completely unknown wallets with no DB record. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
realproject7
added a commit
that referenced
this pull request
Mar 27, 2026
Fix regression: DB-detected owners now correctly use cached agent_wallet for storyline lookup instead of the connected owner wallet address. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
realproject7
added a commit
that referenced
this pull request
Mar 27, 2026
When a user refreshes their profile via /api/user/onboard and has no cached agent_id, check the ERC-8004 registry via getAgentMetadata(). If an agent is found, persist agent columns to DB. This provides the sync path for existing PlotLink users who register as agents externally. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
realproject7
added a commit
that referenced
this pull request
Mar 27, 2026
Profile refresh now checks both agentIdByWallet (bound wallet) and balanceOf + tokenOfOwnerByIndex (NFT owner) before falling back to getAgentMetadataById. Covers existing PlotLink users who register as agents externally with a separate or unset bound wallet. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
realproject7
added a commit
that referenced
this pull request
Mar 27, 2026
getAgentMetadata() now fetches getAgentWallet() in parallel with agentURI and ownerOf, so the agent_wallet field is populated when caching direct bound-wallet registrations via profile refresh. Prevents misclassification of bound wallets as owners. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
text-foreground(white) totext-accent(green) to reinforce the terminal aestheticpx-1topl-[7px] pr-1to align with the front cover's visual left edge (6px layer offset + 1px border)Fixes #294
Test plan
🤖 Generated with Claude Code